Skip to content

[DOCS-10227] VS Code extension revamp #28826

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 17 commits into from
May 13, 2025
Merged

[DOCS-10227] VS Code extension revamp #28826

merged 17 commits into from
May 13, 2025

Conversation

joepeeples
Copy link
Contributor

@joepeeples joepeeples commented Apr 16, 2025 •
edited
Loading

What does this PR do? What is the motivation?

Reorganizes and revises the product docs for Datadog's VS Code extension. Also brings attention to compatibility with Cursor and other forks & flavors.

Merge instructions

Merge readiness:

  • Ready for merge

For Datadog employees:
Merge queue is enabled in this repo. Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). Without this format, your pull request will not pass in CI, the GitLab pipeline will not run, and you won't get a branch preview. Getting a branch preview makes it easier for us to check any issues with your PR, such as broken links.

If your branch doesn't follow this format, rename it or create a new branch and PR.

To have your PR automatically merged after it receives the required reviews, add the following PR comment:

/merge

Additional notes

@joepeeples joepeeples added WORK IN PROGRESS No review needed, it's a wip ;) Do Not Merge Just do not merge this PR :) labels Apr 16, 2025
@joepeeples joepeeples self-assigned this Apr 16, 2025
@github-actions github-actions bot added the Images Images are added/removed with this PR label Apr 16, 2025
@github-actions GitHub Actions
Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@joepeeples
Restore accidentally-deleted file
c1757ca 
Actually doesn't look like we use this image any longer, but I didn't fully confirm this so putting it back just in case...
@joepeeples
Delete code-insights-inline-hover.gif
3cbaf45 
This is what I meant to delete...
@joepeeples
Additional updates
6a1650e 
- Various edits
- Add Cursor to nav menu
- Update some images
- Replace original image (for caching issues): view-in-vscode.png
@joepeeples
Update _index.md
4bafa91
@github-actions github-actions bot added the Architecture Everything related to the Doc backend label Apr 17, 2025
@joepeeples joepeeples requested a review from Vomain April 18, 2025 14:10
Vomain
Vomain reviewed Apr 22, 2025
View reviewed changes
Copy link
Contributor

@Vomain Vomain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for the work @joepeeples! Left some comments here.
FYI some screenshots are outdated or would benefit from better consistency (e.g. same background color and theme), I'll create new ones. I think we should move forward without them though.

@joepeeples joepeeples removed WORK IN PROGRESS No review needed, it's a wip ;) Do Not Merge Just do not merge this PR :) labels May 6, 2025
@joepeeples joepeeples changed the title [DOCS-10227] VS Code extension revamp [WIP] [DOCS-10227] VS Code extension revamp May 6, 2025
@joepeeples joepeeples marked this pull request as ready for review May 6, 2025 14:29
@joepeeples joepeeples requested a review from a team as a code owner May 6, 2025 14:29
@joepeeples joepeeples removed their assignment May 6, 2025
@maycmlee
Copy link
Contributor

maycmlee commented May 6, 2025

Created docs card for review: https://datadoghq.atlassian.net/browse/DOCS-10801

@maycmlee maycmlee added the editorial review Waiting on a more in-depth review label May 6, 2025
@datadog-datadog-prod-us1
Copy link

Datadog Summary

✅ Code Quality    ✅ Code Security    ✅ Dependencies

Was this helpful? Give us feedback!

hestonhoffman
hestonhoffman approved these changes May 13, 2025
View reviewed changes
Copy link
Contributor

@hestonhoffman hestonhoffman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I left one small suggestion. I'm to go ahead and approve despite that suggestion.

@joepeeples
Copy link
Contributor Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented May 13, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-05-13 20:20:43 UTC ℹ️ Start processing command /merge

2025-05-13 20:20:49 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.

2025-05-13 20:50:15 UTC ℹ️ MergeQueue: merge request added to the queue

The expected merge time in master is approximately 20m (p90).

2025-05-13 21:07:05 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue dd-mergequeue bot merged commit a3b5bd4 into master May 13, 2025
17 of 22 checks passed
@dd-mergequeue dd-mergequeue bot deleted the joepeeples/10227-vscode-revamp branch May 13, 2025 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jfree jfree jfree left review comments

@Vomain Vomain Vomain left review comments

@hestonhoffman hestonhoffman hestonhoffman approved these changes

Assignees
No one assigned
Labels
Architecture Everything related to the Doc backend editorial review Waiting on a more in-depth review Images Images are added/removed with this PR mergequeue-status: done
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@joepeeples @maycmlee @jfree @Vomain @hestonhoffman